Skip to content

docs(mesh): same-namespace consumer precedence#5282

Merged
lmilan merged 5 commits into
mainfrom
mesh-same-namespace-exception
May 21, 2026
Merged

docs(mesh): same-namespace consumer precedence#5282
lmilan merged 5 commits into
mainfrom
mesh-same-namespace-exception

Conversation

@slonka
Copy link
Copy Markdown
Contributor

@slonka slonka commented May 19, 2026

Motivation

preview: https://deploy-preview-5282--kongdeveloper.netlify.app/mesh/policies-introduction/#same-namespace-exception

fix: kumahq/kuma#13576

On Kubernetes, a consumer default in the same namespace can still override a producer policy for a specific service because policy role is evaluated before spec.to[].targetRef specificity. The kuma-website docs picked this up but pages under _src/policies/ are no longer auto-synced to developer.konghq.com, so the same note has to be ported by hand to app/mesh/policies-introduction.md.

Implementation information

  • Adds a <sup>note</sup> link from the producer-policies paragraph to the new section.
  • Adds a Same-namespace exception section under policy ordering with Producer / Consumer default / Updated consumer policy navtabs and a short fix-it paragraph.

Supporting documentation

Document that a consumer default in the same namespace can
override a producer policy for a specific service.

Signed-off-by: slonka <slonka@users.noreply.github.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented May 19, 2026

Deploy Preview for kongdeveloper ready!

Name Link
🔨 Latest commit 883b073
🔍 Latest deploy log https://app.netlify.com/projects/kongdeveloper/deploys/6a0ec368bb5c400008a50bfd
😎 Deploy Preview https://deploy-preview-5282--kongdeveloper.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@lmilan lmilan self-requested a review May 19, 2026 11:05
@lmilan lmilan self-assigned this May 19, 2026
The navtabs block requires a tab_group argument.

Signed-off-by: slonka <slonka@users.noreply.github.com>
@slonka slonka marked this pull request as ready for review May 19, 2026 11:32
@slonka slonka requested a review from a team as a code owner May 19, 2026 11:32
Copilot AI review requested due to automatic review settings May 19, 2026 11:32
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Ports the “same-namespace consumer default can override producer policy” nuance from kuma-website into the Mesh policy introduction doc, so Kubernetes users understand why a producer policy may not win when a consumer default exists in the same namespace.

Changes:

  • Adds a superscript cross-link from the Producer policies section to the new exception note.
  • Adds a new “Same-namespace exception” subsection under policy ordering, including tabbed YAML examples and mitigation guidance.

Comment thread app/mesh/policies-introduction.md Outdated
Comment thread app/mesh/policies-introduction.md
slonka and others added 2 commits May 19, 2026 14:47
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@slonka
Copy link
Copy Markdown
Contributor Author

slonka commented May 20, 2026

@lmilan - I've updated the branch so that CI kicks again. Could you review this and get this merged if everything is ok?

@lmilan
Copy link
Copy Markdown
Contributor

lmilan commented May 21, 2026

@slonka I reviewed this and made some changes:

  • Moved the new section to the bottom of the "How policies are combined" section
  • Replaced the <sup> with a note with a link to the section to make it clearer (and because we should avoid using html in md files as much as possible)

Can you take a look and let me know what you think? Thanks!

Copy link
Copy Markdown
Contributor Author

@slonka slonka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@slonka
Copy link
Copy Markdown
Contributor Author

slonka commented May 21, 2026

@lmilan looks better, thanks! can you approve this one and merge it?

@lmilan lmilan merged commit 4401b75 into main May 21, 2026
17 checks passed
@lmilan lmilan deleted the mesh-same-namespace-exception branch May 21, 2026 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix ordering of namespace scoped policies

4 participants